Skip to content

tests/lapi: table tests #142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 28, 2025
Merged

tests/lapi: table tests #142

merged 1 commit into from
Jul 28, 2025
+401 −0

Conversation

ligurio
Copy link
Owner

@ligurio ligurio commented Jul 3, 2025

No description provided.

@ligurio ligurio mentioned this pull request Jul 3, 2025
Closed
7 tasks
@ligurio ligurio marked this pull request as draft July 3, 2025 11:39
@ligurio ligurio force-pushed the ligurio/gh-xxxx-lapi-table branch 4 times, most recently from 4b7cde9 to be50402 Compare July 14, 2025 19:43
@ligurio ligurio marked this pull request as ready for review July 14, 2025 19:48
@ligurio ligurio requested a review from Buristan July 14, 2025 19:48
@ligurio ligurio force-pushed the ligurio/gh-xxxx-lapi-table branch 3 times, most recently from 2791879 to 259b7c2 Compare July 15, 2025 10:06
Buristan
Buristan requested changes Jul 16, 2025
View reviewed changes
Copy link
Collaborator

@Buristan Buristan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, Sergey!
Thanks for the patch!
Please consider my comments below.

@Buristan Buristan assigned ligurio and unassigned Buristan Jul 17, 2025
@ligurio ligurio force-pushed the ligurio/gh-xxxx-lapi-table branch 4 times, most recently from babbc83 to 98a0fe3 Compare July 24, 2025 12:10
@ligurio ligurio assigned Buristan and unassigned ligurio Jul 24, 2025
@ligurio ligurio requested a review from Buristan July 24, 2025 12:10
@ligurio ligurio force-pushed the ligurio/gh-xxxx-lapi-table branch 3 times, most recently from 244864b to 9ecb921 Compare July 24, 2025 15:30
Buristan
Buristan requested changes Jul 25, 2025
View reviewed changes
Copy link
Collaborator

@Buristan Buristan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, Sergey!
Thanks for the fixes!
Please consider bunch of nits below.

@Buristan Buristan assigned ligurio and unassigned Buristan Jul 25, 2025
@ligurio ligurio force-pushed the ligurio/gh-xxxx-lapi-table branch from 9ecb921 to 88d4268 Compare July 25, 2025 10:16
@ligurio
tests/lapi: add table tests
cd6f54b 
The patch adds a fuzzing tests for Lua table functions except the
the function `table.unpack`, it will be tested by
`builtin_unpack_test.lua`.
@ligurio ligurio force-pushed the ligurio/gh-xxxx-lapi-table branch from 88d4268 to cd6f54b Compare July 25, 2025 10:17
Buristan
Buristan approved these changes Jul 25, 2025
View reviewed changes
Copy link
Collaborator

@Buristan Buristan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sergey,
Thanks for the fixes!
LGTM!

@ligurio ligurio mentioned this pull request Jul 25, 2025
Merged
ligurio added a commit to ligurio/lua-c-api-corpus that referenced this pull request Jul 28, 2025
@ligurio
Add seed corpus for table functions
2dafc6f 
Follows up ligurio/lua-c-api-tests#142
Related to tarantool/tarantool#11470
@ligurio ligurio mentioned this pull request Jul 28, 2025
Merged
@ligurio ligurio merged commit ee032e7 into master Jul 28, 2025
12 checks passed
@ligurio ligurio deleted the ligurio/gh-xxxx-lapi-table branch July 28, 2025 11:15
ligurio added a commit to ligurio/lua-c-api-corpus that referenced this pull request Jul 28, 2025
@ligurio
Add seed corpus for table functions
e87a759 
Follows up ligurio/lua-c-api-tests#142
Related to tarantool/tarantool#11470
ligurio added a commit that referenced this pull request Aug 28, 2025
@ligurio
tests/lapi: add table.clear test
603ec82 
The patch adds a test for `table.clear()` function that is
present in LuaJIT. Follows up the commit ee032e7
("tests/lapi: add table tests").

Follows up #142
ligurio added a commit that referenced this pull request Aug 29, 2025
@ligurio
tests/lapi: add table.clear test
54a7ab7 
The patch adds a test for `table.clear()` function that is
present in LuaJIT. Follows up the commit ee032e7
("tests/lapi: add table tests").

Follows up #142
ligurio added a commit that referenced this pull request Aug 29, 2025
@ligurio
tests/lapi: add table.clear test
35b6dc8 
The patch adds a test for `table.clear()` function that is
present in LuaJIT. Follows up the commit ee032e7
("tests/lapi: add table tests").

Follows up #142
ligurio added a commit that referenced this pull request Aug 29, 2025
@ligurio
tests/lapi: add table.clear test
535a0b6 
The patch adds a test for `table.clear()` function that is
present in LuaJIT. Follows up the commit ee032e7
("tests/lapi: add table tests").

Follows up #142
ligurio added a commit to ligurio/lua-c-api-corpus that referenced this pull request Sep 11, 2025
@ligurio
Add seed corpus for table functions (LuaJIT)
c11bb1d 
Follows up ligurio/lua-c-api-tests#142
Related to tarantool/tarantool#11470
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Buristan Buristan Buristan approved these changes

Assignees

@ligurio ligurio

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ligurio @Buristan